Skip to content

Add HGLRCF435AIO config.h #952

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
ot0tot wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add HGLRCF435AIO config.h #952

ot0tot wants to merge 2 commits into from

Conversation

@ot0tot
Copy link
Contributor

@ot0tot ot0tot commented Nov 7, 2025 •
edited by haslinghuis
Loading

Checklist (✓/✕, or y/n)

  • passed Betaflight team's schematics review
  • passed hardware samples testing
  • follows guidelines
  • follows connector standards
  • flight tested
  • comments/issues resolved

Summary by CodeRabbit

  • New Features
    • Added support for the HGLRC F435 AIO flight controller (AT32F435G).
    • Enables onboard gyro/accelerometer, dual barometer options, flash-based Blackbox logging, and gyro clock-in.
    • Preconfigured motor outputs, UARTs, I2C/SPI peripherals, timer mappings, LED strip and beeper (inverted), and ADC-based voltage/current sensing.
    • DShot output enabled by default; default board yaw alignment and system clock preset applied.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 7, 2025 •
edited
Loading

Walkthrough

Adds a new board configuration header for HGLRCF435AIO defining MCU (AT32F435G), board identifiers, enabled peripherals/sensors (ICM42688P, M25P16, BMP280/DPS310), comprehensive pin mappings (motors, UARTs, SPI, I2C, ADC, LEDs, beeper), TIMER_PIN_MAPPING, and default device presets.

Changes

Cohort / File(s) Change Summary
HGLRCF435AIO Hardware Configuration
configs/HGLRCF435AIO/config.h		 New board config with ~60+ macro definitions: MCU/board/manufacturer IDs, USE_* enables (ACC/GYRO/FLASH/BARO), sensor selections (ICM42688P, M25P16, BMP280, DPS310), motor/UART/I2C/SPI/CS/EXTI/CLKIN pin mappings, ADC pins, LED/beeper/LED-strip pins, TIMER_PIN_MAPPING for motors/LEDs/gyro clock, peripheral instance selections, and default device settings (BLACKBOX_FLASH, DSHOT_BITBANG_ON, ADC meters).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

  • Verify pin assignments vs schematic (motor pins, UARTs, SPI/I2C/ADC)
  • Validate TIMER_PIN_MAPPING and timer channel assignments
  • Confirm peripheral instance selections and default device macros

Possibly related PRs

Suggested reviewers

  • nerdCopter

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description only includes a checklist without the required narrative context, objectives, or details about the hardware configuration being added. Add comprehensive description including: hardware overview (MCU, sensors, target specifications), rationale for the new configuration, brief explanation of the changes made, and any relevant testing or validation details beyond the checklist.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Add HGLRCF435AIO config.h' directly and clearly summarizes the main change: adding a new configuration file for the HGLRCF435AIO flight controller.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

haslinghuis
haslinghuis approved these changes Nov 7, 2025
View reviewed changes
@sugaarK sugaarK added the Partner Program this pr is for a target from the partner program and will automatically get tier 1 status label Dec 18, 2025
@HGLRC-T
Copy link
Contributor

HGLRC-T commented Jan 4, 2026 •
edited
Loading

LGTM, So could we merge it? @ot0tot

@HGLRC-T
Copy link
Contributor

HGLRC-T commented Jan 10, 2026

Any idea?

@ot0tot
Copy link
Contributor Author

ot0tot commented Jan 12, 2026

Have you supplied hardware samples?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@haslinghuis haslinghuis haslinghuis approved these changes

@coderabbitai coderabbitai[bot] coderabbitai[bot] approved these changes

+1 more reviewer

@HGLRC-T HGLRC-T HGLRC-T left review comments

Reviewers whose approvals may not affect merge requirements

At least 3 approving reviews are required to merge this pull request.

Assignees

@ot0tot ot0tot

Labels

Awaiting Prototype Samples Awaiting prototype samples Partner Program this pr is for a target from the partner program and will automatically get tier 1 status Schematics Approved Uses Betaflight Connector Standard

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ot0tot @HGLRC-T @haslinghuis @sugaarK